-
-
Notifications
You must be signed in to change notification settings - Fork 106
Update observables + AD tests with MTK v10 #1055
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
If you change the vector ordering you need to change back the answer ordering though 😅 |
We take the output of |
the order of this is not based on the tunable order, it's based on the sort order of |
I had pushed 470880e to fix that. |
Co-authored-by: Christopher Rackauckas <accounts@chrisrackauckas.com>
There seems to be a new failure in SciMLSensitivity/Core1 that I haven't seen earlier. But that is independent of this PR, since this only touches different files, from a different test set |
@ChrisRackauckas |
@AayushSabharwal can take it separately from here. |
Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
There seem to be some variable map changes in latest MTK, so updating tests to reflect that.
Add any other context about the problem here.